Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

merge from master #2

Merged
merged 5 commits into from
Oct 29, 2019
Merged

merge from master #2

merged 5 commits into from
Oct 29, 2019

Conversation

zburning
Copy link
Owner

Description

(Brief description on what this PR is about)

Checklist

Essentials

  • PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage
  • Code is well-documented

Changes

  • Feature1, tests, (and when applicable, API doc)
  • Feature2, tests, (and when applicable, API doc)

Comments

  • If this change is a backward incompatible change, why must this change be made.
  • Interesting edge cases to note here

cc @dmlc/gluon-nlp-team

emilmont and others added 5 commits October 24, 2019 22:01
* Implement: Add raw attention scores to the AttentionCell #951

* Fix pylint issues.

* Address code-review comments.

* Separate _compute_score form _compute_weight.
…vocations (#980)

Test changes
- Enforce that no unexpected warnings are thrown.

API changes
- Update gluonnlp.data.UnigramCandidateSampler to make use of MXNet random.uniform_like operator and drop constructor shape argument. Now the candidates_like argument during forward is used to specify shape of to be sampled candidates. This was the originally intended design, but had to wait for MXNet random.uniform_like operator being available in stable MXNet.
* Update MXNet dependency to cu100-1.6.0b20191027 for CPU and GPU

Nightly release pipelines for cu101-mkl is currently blocked.
Recent CPU builds also failed.

* Simplify Dockerfile

* Fix lint
@zburning zburning merged commit f49b536 into zburning:master Oct 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants